Added flag to strip metadata #281
Conversation
Should we consider removing these infos as the default option? Because I remember you once said that rimage would remove those infos automatically due to privacy concerns. |
This PR is applicable only for ICC and EXIF metadata. I don't see any issues with privacy in that. All other metadata are getting stripped |
There was a problem hiding this comment.
Could not prevent exif info from removing by default setting.
Situation 1: png -> moz
./rimage moz ./exif.png -s "_after"
Situation 2: png -> jpeg
./rimage jpeg ./exif.png -s "_after"
Failed without icc tag and exif infos
Situation 3: png -> png
./rimage png ./exif.png -s "_after"
Failed without icc tag and exif infos
Situation 4: jpg -> png
./rimage png ./color.jpg -s "_after"
Success, but exif infos are missing and the icc info would be cov to rgb (the original icc tag would be removed by default)
Situation 5: jpg -> jpg
./rimage png ./color.jpg -s "_after"
Success, but exif infos are missing and the icc info would be cov to rgb (the original icc tag would be removed by default)
@Mikachu2333 hi, sorry for the long time that I didn't work on this project, this would be the last PR before release of v0.11.0. Please check that metadata (ICC, EXIF) are preserved by default for supported codecs and images are not corrupted